Skip to content
This repository was archived by the owner on Aug 18, 2026. It is now read-only.

Add missing STL stack include - #904

Open
Nil Geisweiller (ngeiswei) wants to merge 1 commit into
microsoft:masterfrom
ngeiswei:fix-missing-stl-include
Open

Nil Geisweiller (ngeiswei) wants to merge 1 commit into
microsoft:masterfrom
ngeiswei:fix-missing-stl-include

Conversation

@ngeiswei

Copy link
Copy Markdown

Otherwise, on gcc 10.2.0, it complains with the error message:

In file included from /home/nilg/MalmoPlatform/Malmo/src/VideoFrameWriter.cpp:27:
/home/nilg/MalmoPlatform/Malmo/src/PosixFrameWriter.h:48:21: error: ‘stack’ in namespace ‘std’ does not name a template type
   48 |         static std::stack<pid_fd> child_process_stack;
      |                     ^~~~~
In file included from /home/nilg/MalmoPlatform/Malmo/src/VideoFrameWriter.cpp:27:
/home/nilg/MalmoPlatform/Malmo/src/PosixFrameWriter.h:1:1: note: ‘std::stack’ is defined in header ‘<stack>’; did you forget to ‘#include <stack>’?
  +++ |+#include <stack>
    1 | // --------------------------------------------------------------------------------------------------

Otherwise, on gcc 10.2.0, it complains with the error message:

```
In file included from /home/nilg/MalmoPlatform/Malmo/src/VideoFrameWriter.cpp:27:
/home/nilg/MalmoPlatform/Malmo/src/PosixFrameWriter.h:48:21: error: ‘stack’ in namespace ‘std’ does not name a template type
   48 |         static std::stack<pid_fd> child_process_stack;
      |                     ^~~~~
In file included from /home/nilg/MalmoPlatform/Malmo/src/VideoFrameWriter.cpp:27:
/home/nilg/MalmoPlatform/Malmo/src/PosixFrameWriter.h:1:1: note: ‘std::stack’ is defined in header ‘<stack>’; did you forget to ‘#include <stack>’?
  +++ |+#include <stack>
    1 | // --------------------------------------------------------------------------------------------------
```

This branch has not been deployed

No deployments
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant